Skip to content

fix(transactions): Defer dynamic sampling until metrics config is valid#6246

Merged
Dav1dde merged 9 commits into
masterfrom
fix/ds-no-metrics
Jul 24, 2026
Merged

fix(transactions): Defer dynamic sampling until metrics config is valid#6246
Dav1dde merged 9 commits into
masterfrom
fix/ds-no-metrics

Conversation

@jjbayer

@jjbayer jjbayer commented Jul 17, 2026

Copy link
Copy Markdown
Member

If metrics extraction config is invalid, delegate dynamic sampling to the next relay. This prevents dropping transactions for which no metrics were extracted yet.

Fixes RELAY-268

@linear-code

linear-code Bot commented Jul 17, 2026

Copy link
Copy Markdown

RELAY-268

Comment thread relay-server/src/processing/transactions/process.rs Outdated
Comment thread relay-server/src/processing/transactions/types/expanded.rs Outdated
@jjbayer
jjbayer marked this pull request as ready for review July 17, 2026 11:51
@jjbayer
jjbayer requested a review from a team as a code owner July 17, 2026 11:51
Comment thread relay-server/src/processing/transactions/process.rs Outdated
@Dav1dde
Dav1dde force-pushed the fix/ds-no-metrics branch 2 times, most recently from c672f7a to 4537d8e Compare July 24, 2026 07:29
@Dav1dde
Dav1dde force-pushed the fix/ds-no-metrics branch from 4537d8e to 622557e Compare July 24, 2026 07:29
cursor[bot]

This comment was marked as resolved.

@Dav1dde
Dav1dde enabled auto-merge July 24, 2026 07:33
@Dav1dde
Dav1dde force-pushed the fix/ds-no-metrics branch from 5dd2140 to bfda6e4 Compare July 24, 2026 07:38
cursor[bot]

This comment was marked as low quality.

@Dav1dde
Dav1dde added this pull request to the merge queue Jul 24, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 24, 2026
@Dav1dde
Dav1dde added this pull request to the merge queue Jul 24, 2026
Merged via the queue into master with commit 3fabec1 Jul 24, 2026
35 checks passed
@Dav1dde
Dav1dde deleted the fix/ds-no-metrics branch July 24, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants